make this somewhat key-navigable
authorHavoc Pennington <hp@pobox.com>
Sat, 24 Mar 2001 06:10:40 +0000 (06:10 +0000)
committerHavoc Pennington <hp@src.gnome.org>
Sat, 24 Mar 2001 06:10:40 +0000 (06:10 +0000)
2001-03-23  Havoc Pennington  <hp@pobox.com>

* gtk/gtkhsv.c: make this somewhat key-navigable

* gtk/gtkfontsel.c: mnemonics

* gtk/gtkgamma.c: mnemonics

* gtk/gtkcolorsel.c: add mnemonics to labels

2001-03-22  Havoc Pennington  <hp@pobox.com>

Applied big patch from Seth Lytle to fix event handler return
values, slightly modified.

* gtk/gtkcalendar.c (gtk_calendar_button_press): return TRUE if
handled.
(gtk_calendar_key_press): return TRUE for GDK_space triggers
select_day

* gtk/gtkclist.c (gtk_clist_button_press): return TRUE if handled
(gtk_clist_button_release): return TRUE if handled

* gtk/gtkcolorsel.c (mouse_release):
(palette_activate):
(palette_press):
(palette_new):
(mouse_press):
(get_screen_color):
fixed return type (void to gboolean), return TRUE,
added GTK_SIGNAL_FUNC cast

* gtk/gtkcombo.c (gtk_combo_popup_button_press): fixed return type
(void to gboolean), and return TRUE
(gtk_combo_button_release): changed return value to TRUE after
gtk_grab_add on combo->popwin

* gtk/gtkctree.c (gtk_ctree_button_press): return TRUE if triggers a
collapse or expand

* gtk/gtkcurve.c (gtk_curve_graph_events): return TRUE for
button/motion

* gtk/gtkdnd.c (gtk_drag_source_event_cb): return TRUE for button
press/release and motion that trigger act

* gtk/gtkentry.c (gtk_entry_button_release): return TRUE

* gtk/gtkfontsel.c (gtk_font_selection_size_key_press): return TRUE
for GDK_Return

* gtk/gtklist.c (gtk_list_button_press): return TRUE mostly
(gtk_list_button_release): return TRUE if selection

* gtk/gtklistitem.c (gtk_list_item_button_press): return TRUE on
press

* gtk/gtkmenushell.c (gtk_menu_shell_button_press):
(gtk_menu_shell_button_release): chain parent_menu_shell retval

* gtk/gtknotebook.c (gtk_notebook_button_press): return FALSE if
nothing triggered
(gtk_notebook_button_release): return TRUE if we do anything

* gtk/gtkspinbutton.c (gtk_spin_button_button_press):
(gtk_spin_button_button_release): return TRUE or chained retval

* gtk/gtktext.c (gtk_text_button_press):
(gtk_text_button_release): return TRUE
(gtk_text_key_press): remove redundant retval code

* gtk/gtktreeitem.c (gtk_tree_item_button_press): return TRUE if button
and sensitive
(gtk_tree_item_subtree_button_click): changed type to gint, and return
TRUE if sensitive

* gtk/gtktooltips.c (gtk_tooltips_event_handler): deactivate implies
return TRUE

34 files changed:
ChangeLog
ChangeLog.pre-2-0
ChangeLog.pre-2-10
ChangeLog.pre-2-2
ChangeLog.pre-2-4
ChangeLog.pre-2-6
ChangeLog.pre-2-8
gtk/gtkcalendar.c
gtk/gtkcellrenderertoggle.c
gtk/gtkclist.c
gtk/gtkcolorsel.c
gtk/gtkcombo.c
gtk/gtkctree.c
gtk/gtkcurve.c
gtk/gtkdnd.c
gtk/gtkentry.c
gtk/gtkfilesel.c
gtk/gtkfontsel.c
gtk/gtkgamma.c
gtk/gtkhsv.c
gtk/gtkhsv.h
gtk/gtkinputdialog.c
gtk/gtklist.c
gtk/gtklistitem.c
gtk/gtkmenushell.c
gtk/gtknotebook.c
gtk/gtkspinbutton.c
gtk/gtktext.c
gtk/gtktexttag.c
gtk/gtktextview.c
gtk/gtktooltips.c
gtk/gtktreeitem.c
gtk/gtktreemodelsort.c
gtk/gtktreeview.c

index 2cf93341e228a9f40527f9ebc8434d5320f4ac30..4a9a13e5b07fbf38d1745aa9ee7435deea5f7d55 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,82 @@
+2001-03-23  Havoc Pennington  <hp@pobox.com>
+
+       * gtk/gtkhsv.c: make this somewhat key-navigable
+
+       * gtk/gtkfontsel.c: mnemonics
+
+       * gtk/gtkgamma.c: mnemonics
+
+       * gtk/gtkcolorsel.c: add mnemonics to labels
+
+2001-03-22  Havoc Pennington  <hp@pobox.com>
+
+       Applied big patch from Seth Lytle to fix event handler return
+       values, slightly modified.
+
+       * gtk/gtkcalendar.c (gtk_calendar_button_press): return TRUE if
+       handled.
+       (gtk_calendar_key_press): return TRUE for GDK_space triggers
+       select_day
+
+       * gtk/gtkclist.c (gtk_clist_button_press): return TRUE if handled
+       (gtk_clist_button_release): return TRUE if handled
+
+       * gtk/gtkcolorsel.c (mouse_release):
+       (palette_activate): 
+       (palette_press): 
+       (palette_new): 
+       (mouse_press): 
+       (get_screen_color): 
+       fixed return type (void to gboolean), return TRUE,
+       added GTK_SIGNAL_FUNC cast
+
+       * gtk/gtkcombo.c (gtk_combo_popup_button_press): fixed return type
+       (void to gboolean), and return TRUE
+       (gtk_combo_button_release): changed return value to TRUE after
+       gtk_grab_add on combo->popwin
+
+       * gtk/gtkctree.c (gtk_ctree_button_press): return TRUE if triggers a
+       collapse or expand
+
+       * gtk/gtkcurve.c (gtk_curve_graph_events): return TRUE for
+       button/motion
+
+       * gtk/gtkdnd.c (gtk_drag_source_event_cb): return TRUE for button
+       press/release and motion that trigger act
+
+       * gtk/gtkentry.c (gtk_entry_button_release): return TRUE
+
+       * gtk/gtkfontsel.c (gtk_font_selection_size_key_press): return TRUE
+       for GDK_Return
+       
+       * gtk/gtklist.c (gtk_list_button_press): return TRUE mostly
+       (gtk_list_button_release): return TRUE if selection
+
+       * gtk/gtklistitem.c (gtk_list_item_button_press): return TRUE on
+       press
+
+       * gtk/gtkmenushell.c (gtk_menu_shell_button_press):
+       (gtk_menu_shell_button_release): chain parent_menu_shell retval
+
+       * gtk/gtknotebook.c (gtk_notebook_button_press): return FALSE if
+       nothing triggered
+       (gtk_notebook_button_release): return TRUE if we do anything
+
+       * gtk/gtkspinbutton.c (gtk_spin_button_button_press): 
+       (gtk_spin_button_button_release): return TRUE or chained retval
+
+       * gtk/gtktext.c (gtk_text_button_press): 
+       (gtk_text_button_release): return TRUE
+       (gtk_text_key_press): remove redundant retval code
+       
+       * gtk/gtktreeitem.c (gtk_tree_item_button_press): return TRUE if button
+       and sensitive
+       (gtk_tree_item_subtree_button_click): changed type to gint, and return
+       TRUE if sensitive
+
+       * gtk/gtktooltips.c (gtk_tooltips_event_handler): deactivate implies 
+       return TRUE
+
 2001-03-23  Havoc Pennington  <hp@redhat.com>
 
        * gtk/testgtk.c: fix warnings
index 2cf93341e228a9f40527f9ebc8434d5320f4ac30..4a9a13e5b07fbf38d1745aa9ee7435deea5f7d55 100644 (file)
@@ -1,3 +1,82 @@
+2001-03-23  Havoc Pennington  <hp@pobox.com>
+
+       * gtk/gtkhsv.c: make this somewhat key-navigable
+
+       * gtk/gtkfontsel.c: mnemonics
+
+       * gtk/gtkgamma.c: mnemonics
+
+       * gtk/gtkcolorsel.c: add mnemonics to labels
+
+2001-03-22  Havoc Pennington  <hp@pobox.com>
+
+       Applied big patch from Seth Lytle to fix event handler return
+       values, slightly modified.
+
+       * gtk/gtkcalendar.c (gtk_calendar_button_press): return TRUE if
+       handled.
+       (gtk_calendar_key_press): return TRUE for GDK_space triggers
+       select_day
+
+       * gtk/gtkclist.c (gtk_clist_button_press): return TRUE if handled
+       (gtk_clist_button_release): return TRUE if handled
+
+       * gtk/gtkcolorsel.c (mouse_release):
+       (palette_activate): 
+       (palette_press): 
+       (palette_new): 
+       (mouse_press): 
+       (get_screen_color): 
+       fixed return type (void to gboolean), return TRUE,
+       added GTK_SIGNAL_FUNC cast
+
+       * gtk/gtkcombo.c (gtk_combo_popup_button_press): fixed return type
+       (void to gboolean), and return TRUE
+       (gtk_combo_button_release): changed return value to TRUE after
+       gtk_grab_add on combo->popwin
+
+       * gtk/gtkctree.c (gtk_ctree_button_press): return TRUE if triggers a
+       collapse or expand
+
+       * gtk/gtkcurve.c (gtk_curve_graph_events): return TRUE for
+       button/motion
+
+       * gtk/gtkdnd.c (gtk_drag_source_event_cb): return TRUE for button
+       press/release and motion that trigger act
+
+       * gtk/gtkentry.c (gtk_entry_button_release): return TRUE
+
+       * gtk/gtkfontsel.c (gtk_font_selection_size_key_press): return TRUE
+       for GDK_Return
+       
+       * gtk/gtklist.c (gtk_list_button_press): return TRUE mostly
+       (gtk_list_button_release): return TRUE if selection
+
+       * gtk/gtklistitem.c (gtk_list_item_button_press): return TRUE on
+       press
+
+       * gtk/gtkmenushell.c (gtk_menu_shell_button_press):
+       (gtk_menu_shell_button_release): chain parent_menu_shell retval
+
+       * gtk/gtknotebook.c (gtk_notebook_button_press): return FALSE if
+       nothing triggered
+       (gtk_notebook_button_release): return TRUE if we do anything
+
+       * gtk/gtkspinbutton.c (gtk_spin_button_button_press): 
+       (gtk_spin_button_button_release): return TRUE or chained retval
+
+       * gtk/gtktext.c (gtk_text_button_press): 
+       (gtk_text_button_release): return TRUE
+       (gtk_text_key_press): remove redundant retval code
+       
+       * gtk/gtktreeitem.c (gtk_tree_item_button_press): return TRUE if button
+       and sensitive
+       (gtk_tree_item_subtree_button_click): changed type to gint, and return
+       TRUE if sensitive
+
+       * gtk/gtktooltips.c (gtk_tooltips_event_handler): deactivate implies 
+       return TRUE
+
 2001-03-23  Havoc Pennington  <hp@redhat.com>
 
        * gtk/testgtk.c: fix warnings
index 2cf93341e228a9f40527f9ebc8434d5320f4ac30..4a9a13e5b07fbf38d1745aa9ee7435deea5f7d55 100644 (file)
@@ -1,3 +1,82 @@
+2001-03-23  Havoc Pennington  <hp@pobox.com>
+
+       * gtk/gtkhsv.c: make this somewhat key-navigable
+
+       * gtk/gtkfontsel.c: mnemonics
+
+       * gtk/gtkgamma.c: mnemonics
+
+       * gtk/gtkcolorsel.c: add mnemonics to labels
+
+2001-03-22  Havoc Pennington  <hp@pobox.com>
+
+       Applied big patch from Seth Lytle to fix event handler return
+       values, slightly modified.
+
+       * gtk/gtkcalendar.c (gtk_calendar_button_press): return TRUE if
+       handled.
+       (gtk_calendar_key_press): return TRUE for GDK_space triggers
+       select_day
+
+       * gtk/gtkclist.c (gtk_clist_button_press): return TRUE if handled
+       (gtk_clist_button_release): return TRUE if handled
+
+       * gtk/gtkcolorsel.c (mouse_release):
+       (palette_activate): 
+       (palette_press): 
+       (palette_new): 
+       (mouse_press): 
+       (get_screen_color): 
+       fixed return type (void to gboolean), return TRUE,
+       added GTK_SIGNAL_FUNC cast
+
+       * gtk/gtkcombo.c (gtk_combo_popup_button_press): fixed return type
+       (void to gboolean), and return TRUE
+       (gtk_combo_button_release): changed return value to TRUE after
+       gtk_grab_add on combo->popwin
+
+       * gtk/gtkctree.c (gtk_ctree_button_press): return TRUE if triggers a
+       collapse or expand
+
+       * gtk/gtkcurve.c (gtk_curve_graph_events): return TRUE for
+       button/motion
+
+       * gtk/gtkdnd.c (gtk_drag_source_event_cb): return TRUE for button
+       press/release and motion that trigger act
+
+       * gtk/gtkentry.c (gtk_entry_button_release): return TRUE
+
+       * gtk/gtkfontsel.c (gtk_font_selection_size_key_press): return TRUE
+       for GDK_Return
+       
+       * gtk/gtklist.c (gtk_list_button_press): return TRUE mostly
+       (gtk_list_button_release): return TRUE if selection
+
+       * gtk/gtklistitem.c (gtk_list_item_button_press): return TRUE on
+       press
+
+       * gtk/gtkmenushell.c (gtk_menu_shell_button_press):
+       (gtk_menu_shell_button_release): chain parent_menu_shell retval
+
+       * gtk/gtknotebook.c (gtk_notebook_button_press): return FALSE if
+       nothing triggered
+       (gtk_notebook_button_release): return TRUE if we do anything
+
+       * gtk/gtkspinbutton.c (gtk_spin_button_button_press): 
+       (gtk_spin_button_button_release): return TRUE or chained retval
+
+       * gtk/gtktext.c (gtk_text_button_press): 
+       (gtk_text_button_release): return TRUE
+       (gtk_text_key_press): remove redundant retval code
+       
+       * gtk/gtktreeitem.c (gtk_tree_item_button_press): return TRUE if button
+       and sensitive
+       (gtk_tree_item_subtree_button_click): changed type to gint, and return
+       TRUE if sensitive
+
+       * gtk/gtktooltips.c (gtk_tooltips_event_handler): deactivate implies 
+       return TRUE
+
 2001-03-23  Havoc Pennington  <hp@redhat.com>
 
        * gtk/testgtk.c: fix warnings
index 2cf93341e228a9f40527f9ebc8434d5320f4ac30..4a9a13e5b07fbf38d1745aa9ee7435deea5f7d55 100644 (file)
@@ -1,3 +1,82 @@
+2001-03-23  Havoc Pennington  <hp@pobox.com>
+
+       * gtk/gtkhsv.c: make this somewhat key-navigable
+
+       * gtk/gtkfontsel.c: mnemonics
+
+       * gtk/gtkgamma.c: mnemonics
+
+       * gtk/gtkcolorsel.c: add mnemonics to labels
+
+2001-03-22  Havoc Pennington  <hp@pobox.com>
+
+       Applied big patch from Seth Lytle to fix event handler return
+       values, slightly modified.
+
+       * gtk/gtkcalendar.c (gtk_calendar_button_press): return TRUE if
+       handled.
+       (gtk_calendar_key_press): return TRUE for GDK_space triggers
+       select_day
+
+       * gtk/gtkclist.c (gtk_clist_button_press): return TRUE if handled
+       (gtk_clist_button_release): return TRUE if handled
+
+       * gtk/gtkcolorsel.c (mouse_release):
+       (palette_activate): 
+       (palette_press): 
+       (palette_new): 
+       (mouse_press): 
+       (get_screen_color): 
+       fixed return type (void to gboolean), return TRUE,
+       added GTK_SIGNAL_FUNC cast
+
+       * gtk/gtkcombo.c (gtk_combo_popup_button_press): fixed return type
+       (void to gboolean), and return TRUE
+       (gtk_combo_button_release): changed return value to TRUE after
+       gtk_grab_add on combo->popwin
+
+       * gtk/gtkctree.c (gtk_ctree_button_press): return TRUE if triggers a
+       collapse or expand
+
+       * gtk/gtkcurve.c (gtk_curve_graph_events): return TRUE for
+       button/motion
+
+       * gtk/gtkdnd.c (gtk_drag_source_event_cb): return TRUE for button
+       press/release and motion that trigger act
+
+       * gtk/gtkentry.c (gtk_entry_button_release): return TRUE
+
+       * gtk/gtkfontsel.c (gtk_font_selection_size_key_press): return TRUE
+       for GDK_Return
+       
+       * gtk/gtklist.c (gtk_list_button_press): return TRUE mostly
+       (gtk_list_button_release): return TRUE if selection
+
+       * gtk/gtklistitem.c (gtk_list_item_button_press): return TRUE on
+       press
+
+       * gtk/gtkmenushell.c (gtk_menu_shell_button_press):
+       (gtk_menu_shell_button_release): chain parent_menu_shell retval
+
+       * gtk/gtknotebook.c (gtk_notebook_button_press): return FALSE if
+       nothing triggered
+       (gtk_notebook_button_release): return TRUE if we do anything
+
+       * gtk/gtkspinbutton.c (gtk_spin_button_button_press): 
+       (gtk_spin_button_button_release): return TRUE or chained retval
+
+       * gtk/gtktext.c (gtk_text_button_press): 
+       (gtk_text_button_release): return TRUE
+       (gtk_text_key_press): remove redundant retval code
+       
+       * gtk/gtktreeitem.c (gtk_tree_item_button_press): return TRUE if button
+       and sensitive
+       (gtk_tree_item_subtree_button_click): changed type to gint, and return
+       TRUE if sensitive
+
+       * gtk/gtktooltips.c (gtk_tooltips_event_handler): deactivate implies 
+       return TRUE
+
 2001-03-23  Havoc Pennington  <hp@redhat.com>
 
        * gtk/testgtk.c: fix warnings
index 2cf93341e228a9f40527f9ebc8434d5320f4ac30..4a9a13e5b07fbf38d1745aa9ee7435deea5f7d55 100644 (file)
@@ -1,3 +1,82 @@
+2001-03-23  Havoc Pennington  <hp@pobox.com>
+
+       * gtk/gtkhsv.c: make this somewhat key-navigable
+
+       * gtk/gtkfontsel.c: mnemonics
+
+       * gtk/gtkgamma.c: mnemonics
+
+       * gtk/gtkcolorsel.c: add mnemonics to labels
+
+2001-03-22  Havoc Pennington  <hp@pobox.com>
+
+       Applied big patch from Seth Lytle to fix event handler return
+       values, slightly modified.
+
+       * gtk/gtkcalendar.c (gtk_calendar_button_press): return TRUE if
+       handled.
+       (gtk_calendar_key_press): return TRUE for GDK_space triggers
+       select_day
+
+       * gtk/gtkclist.c (gtk_clist_button_press): return TRUE if handled
+       (gtk_clist_button_release): return TRUE if handled
+
+       * gtk/gtkcolorsel.c (mouse_release):
+       (palette_activate): 
+       (palette_press): 
+       (palette_new): 
+       (mouse_press): 
+       (get_screen_color): 
+       fixed return type (void to gboolean), return TRUE,
+       added GTK_SIGNAL_FUNC cast
+
+       * gtk/gtkcombo.c (gtk_combo_popup_button_press): fixed return type
+       (void to gboolean), and return TRUE
+       (gtk_combo_button_release): changed return value to TRUE after
+       gtk_grab_add on combo->popwin
+
+       * gtk/gtkctree.c (gtk_ctree_button_press): return TRUE if triggers a
+       collapse or expand
+
+       * gtk/gtkcurve.c (gtk_curve_graph_events): return TRUE for
+       button/motion
+
+       * gtk/gtkdnd.c (gtk_drag_source_event_cb): return TRUE for button
+       press/release and motion that trigger act
+
+       * gtk/gtkentry.c (gtk_entry_button_release): return TRUE
+
+       * gtk/gtkfontsel.c (gtk_font_selection_size_key_press): return TRUE
+       for GDK_Return
+       
+       * gtk/gtklist.c (gtk_list_button_press): return TRUE mostly
+       (gtk_list_button_release): return TRUE if selection
+
+       * gtk/gtklistitem.c (gtk_list_item_button_press): return TRUE on
+       press
+
+       * gtk/gtkmenushell.c (gtk_menu_shell_button_press):
+       (gtk_menu_shell_button_release): chain parent_menu_shell retval
+
+       * gtk/gtknotebook.c (gtk_notebook_button_press): return FALSE if
+       nothing triggered
+       (gtk_notebook_button_release): return TRUE if we do anything
+
+       * gtk/gtkspinbutton.c (gtk_spin_button_button_press): 
+       (gtk_spin_button_button_release): return TRUE or chained retval
+
+       * gtk/gtktext.c (gtk_text_button_press): 
+       (gtk_text_button_release): return TRUE
+       (gtk_text_key_press): remove redundant retval code
+       
+       * gtk/gtktreeitem.c (gtk_tree_item_button_press): return TRUE if button
+       and sensitive
+       (gtk_tree_item_subtree_button_click): changed type to gint, and return
+       TRUE if sensitive
+
+       * gtk/gtktooltips.c (gtk_tooltips_event_handler): deactivate implies 
+       return TRUE
+
 2001-03-23  Havoc Pennington  <hp@redhat.com>
 
        * gtk/testgtk.c: fix warnings
index 2cf93341e228a9f40527f9ebc8434d5320f4ac30..4a9a13e5b07fbf38d1745aa9ee7435deea5f7d55 100644 (file)
@@ -1,3 +1,82 @@
+2001-03-23  Havoc Pennington  <hp@pobox.com>
+
+       * gtk/gtkhsv.c: make this somewhat key-navigable
+
+       * gtk/gtkfontsel.c: mnemonics
+
+       * gtk/gtkgamma.c: mnemonics
+
+       * gtk/gtkcolorsel.c: add mnemonics to labels
+
+2001-03-22  Havoc Pennington  <hp@pobox.com>
+
+       Applied big patch from Seth Lytle to fix event handler return
+       values, slightly modified.
+
+       * gtk/gtkcalendar.c (gtk_calendar_button_press): return TRUE if
+       handled.
+       (gtk_calendar_key_press): return TRUE for GDK_space triggers
+       select_day
+
+       * gtk/gtkclist.c (gtk_clist_button_press): return TRUE if handled
+       (gtk_clist_button_release): return TRUE if handled
+
+       * gtk/gtkcolorsel.c (mouse_release):
+       (palette_activate): 
+       (palette_press): 
+       (palette_new): 
+       (mouse_press): 
+       (get_screen_color): 
+       fixed return type (void to gboolean), return TRUE,
+       added GTK_SIGNAL_FUNC cast
+
+       * gtk/gtkcombo.c (gtk_combo_popup_button_press): fixed return type
+       (void to gboolean), and return TRUE
+       (gtk_combo_button_release): changed return value to TRUE after
+       gtk_grab_add on combo->popwin
+
+       * gtk/gtkctree.c (gtk_ctree_button_press): return TRUE if triggers a
+       collapse or expand
+
+       * gtk/gtkcurve.c (gtk_curve_graph_events): return TRUE for
+       button/motion
+
+       * gtk/gtkdnd.c (gtk_drag_source_event_cb): return TRUE for button
+       press/release and motion that trigger act
+
+       * gtk/gtkentry.c (gtk_entry_button_release): return TRUE
+
+       * gtk/gtkfontsel.c (gtk_font_selection_size_key_press): return TRUE
+       for GDK_Return
+       
+       * gtk/gtklist.c (gtk_list_button_press): return TRUE mostly
+       (gtk_list_button_release): return TRUE if selection
+
+       * gtk/gtklistitem.c (gtk_list_item_button_press): return TRUE on
+       press
+
+       * gtk/gtkmenushell.c (gtk_menu_shell_button_press):
+       (gtk_menu_shell_button_release): chain parent_menu_shell retval
+
+       * gtk/gtknotebook.c (gtk_notebook_button_press): return FALSE if
+       nothing triggered
+       (gtk_notebook_button_release): return TRUE if we do anything
+
+       * gtk/gtkspinbutton.c (gtk_spin_button_button_press): 
+       (gtk_spin_button_button_release): return TRUE or chained retval
+
+       * gtk/gtktext.c (gtk_text_button_press): 
+       (gtk_text_button_release): return TRUE
+       (gtk_text_key_press): remove redundant retval code
+       
+       * gtk/gtktreeitem.c (gtk_tree_item_button_press): return TRUE if button
+       and sensitive
+       (gtk_tree_item_subtree_button_click): changed type to gint, and return
+       TRUE if sensitive
+
+       * gtk/gtktooltips.c (gtk_tooltips_event_handler): deactivate implies 
+       return TRUE
+
 2001-03-23  Havoc Pennington  <hp@redhat.com>
 
        * gtk/testgtk.c: fix warnings
index 2cf93341e228a9f40527f9ebc8434d5320f4ac30..4a9a13e5b07fbf38d1745aa9ee7435deea5f7d55 100644 (file)
@@ -1,3 +1,82 @@
+2001-03-23  Havoc Pennington  <hp@pobox.com>
+
+       * gtk/gtkhsv.c: make this somewhat key-navigable
+
+       * gtk/gtkfontsel.c: mnemonics
+
+       * gtk/gtkgamma.c: mnemonics
+
+       * gtk/gtkcolorsel.c: add mnemonics to labels
+
+2001-03-22  Havoc Pennington  <hp@pobox.com>
+
+       Applied big patch from Seth Lytle to fix event handler return
+       values, slightly modified.
+
+       * gtk/gtkcalendar.c (gtk_calendar_button_press): return TRUE if
+       handled.
+       (gtk_calendar_key_press): return TRUE for GDK_space triggers
+       select_day
+
+       * gtk/gtkclist.c (gtk_clist_button_press): return TRUE if handled
+       (gtk_clist_button_release): return TRUE if handled
+
+       * gtk/gtkcolorsel.c (mouse_release):
+       (palette_activate): 
+       (palette_press): 
+       (palette_new): 
+       (mouse_press): 
+       (get_screen_color): 
+       fixed return type (void to gboolean), return TRUE,
+       added GTK_SIGNAL_FUNC cast
+
+       * gtk/gtkcombo.c (gtk_combo_popup_button_press): fixed return type
+       (void to gboolean), and return TRUE
+       (gtk_combo_button_release): changed return value to TRUE after
+       gtk_grab_add on combo->popwin
+
+       * gtk/gtkctree.c (gtk_ctree_button_press): return TRUE if triggers a
+       collapse or expand
+
+       * gtk/gtkcurve.c (gtk_curve_graph_events): return TRUE for
+       button/motion
+
+       * gtk/gtkdnd.c (gtk_drag_source_event_cb): return TRUE for button
+       press/release and motion that trigger act
+
+       * gtk/gtkentry.c (gtk_entry_button_release): return TRUE
+
+       * gtk/gtkfontsel.c (gtk_font_selection_size_key_press): return TRUE
+       for GDK_Return
+       
+       * gtk/gtklist.c (gtk_list_button_press): return TRUE mostly
+       (gtk_list_button_release): return TRUE if selection
+
+       * gtk/gtklistitem.c (gtk_list_item_button_press): return TRUE on
+       press
+
+       * gtk/gtkmenushell.c (gtk_menu_shell_button_press):
+       (gtk_menu_shell_button_release): chain parent_menu_shell retval
+
+       * gtk/gtknotebook.c (gtk_notebook_button_press): return FALSE if
+       nothing triggered
+       (gtk_notebook_button_release): return TRUE if we do anything
+
+       * gtk/gtkspinbutton.c (gtk_spin_button_button_press): 
+       (gtk_spin_button_button_release): return TRUE or chained retval
+
+       * gtk/gtktext.c (gtk_text_button_press): 
+       (gtk_text_button_release): return TRUE
+       (gtk_text_key_press): remove redundant retval code
+       
+       * gtk/gtktreeitem.c (gtk_tree_item_button_press): return TRUE if button
+       and sensitive
+       (gtk_tree_item_subtree_button_click): changed type to gint, and return
+       TRUE if sensitive
+
+       * gtk/gtktooltips.c (gtk_tooltips_event_handler): deactivate implies 
+       return TRUE
+
 2001-03-23  Havoc Pennington  <hp@redhat.com>
 
        * gtk/testgtk.c: fix warnings
index 05eb0f463a3dc534db9b9559dedb9c938ad5e27b..ad3571400aa09a772bb31c325f5e95414dd1dd66 100644 (file)
@@ -2305,6 +2305,7 @@ gtk_calendar_button_press (GtkWidget        *widget,
   GtkCalendar *calendar;
   GtkCalendarPrivateData *private_data;
   gint x, y;
+  void (* action_func) (GtkCalendar *);
   
   g_return_val_if_fail (widget != NULL, FALSE);
   g_return_val_if_fail (GTK_IS_CALENDAR (widget), FALSE);
@@ -2318,23 +2319,23 @@ gtk_calendar_button_press (GtkWidget      *widget,
   
   if (event->window == private_data->main_win)
     gtk_calendar_main_button (widget, event);
-  
-  if (event->type != GDK_BUTTON_PRESS)
-    return FALSE; /* Double-clicks? Triple-clicks? No thanks! */
+
+  action_func = NULL;  
 
   if (event->window == private_data->arrow_win[ARROW_MONTH_LEFT])
-    gtk_calendar_set_month_prev (calendar);
-  
-  if (event->window == private_data->arrow_win[ARROW_MONTH_RIGHT])
-    gtk_calendar_set_month_next (calendar);
-  
-  if (event->window == private_data->arrow_win[ARROW_YEAR_LEFT])
-    gtk_calendar_set_year_prev (calendar);
-  
-  if (event->window == private_data->arrow_win[ARROW_YEAR_RIGHT])
-    gtk_calendar_set_year_next (calendar);
-  
-  return FALSE;
+    action_func = gtk_calendar_set_month_prev;
+  else if (event->window == private_data->arrow_win[ARROW_MONTH_RIGHT])
+    action_func = gtk_calendar_set_month_next;
+  else if (event->window == private_data->arrow_win[ARROW_YEAR_LEFT])
+    action_func = gtk_calendar_set_year_prev;
+  else if (event->window == private_data->arrow_win[ARROW_YEAR_RIGHT])
+    action_func = gtk_calendar_set_year_next;
+
+  /* only call the action on single click, not double */
+  if (event->type == GDK_BUTTON_PRESS)
+    (* action_func) (calendar);
+  
+  return action_func != NULL;
 }
 
 static gboolean
@@ -2732,6 +2733,7 @@ gtk_calendar_key_press (GtkWidget   *widget,
       
       if (row > -1 && col > -1)
        {
+         return_val = TRUE;
          gtk_calendar_freeze (calendar);         
 
          if (calendar->day_month[row][col] == MONTH_PREV)
index 6207968f391f83c32bb0b0fa8e9a1af620938c53..bf9c421dad2a05748f68a11c764cb0ba65618c54 100644 (file)
@@ -317,7 +317,7 @@ gtk_cell_renderer_toggle_event (GtkCellRenderer *cell,
                                guint            flags)
 {
   GtkCellRendererToggle *celltoggle;
-  gint retval = FALSE;
+  gboolean retval = FALSE;
   
   celltoggle = GTK_CELL_RENDERER_TOGGLE (cell);
   
index c501aba51f74b252b0376ece4934262740ee96ff..bcca0abcbad35a37ac33f54114e209f70f147079 100644 (file)
@@ -5024,7 +5024,7 @@ gtk_clist_button_press (GtkWidget      *widget,
                }
            }
        }
-      return FALSE;
+      return TRUE;
     }
 
   /* press on resize windows */
@@ -5059,7 +5059,10 @@ gtk_clist_button_press (GtkWidget      *widget,
        if (GTK_CLIST_ADD_MODE(clist))
          gdk_gc_set_line_attributes (clist->xor_gc, 1, GDK_LINE_SOLID, 0, 0);
        draw_xor_line (clist);
+
+        return TRUE;
       }
+
   return FALSE;
 }
 
@@ -5113,7 +5116,8 @@ gtk_clist_button_release (GtkWidget      *widget,
 
       width = new_column_width (clist, i, &x);
       gtk_clist_set_column_width (clist, i, width);
-      return FALSE;
+
+      return TRUE;
     }
 
   if (clist->drag_button == event->button)
@@ -5153,7 +5157,10 @@ gtk_clist_button_release (GtkWidget      *widget,
              break;
            }
        }
+
+      return TRUE;
     }
+  
   return FALSE;
 }
 
index a87258895a261a885800f5b78df3e009b597fa2d..0a706f013e2492c207cc9722b01eaef75514bc96 100644 (file)
@@ -598,7 +598,7 @@ palette_expose (GtkWidget      *drawing_area,
   palette_paint (drawing_area, &(event->area), data);
 }
 
-static void
+static gint
 palette_press (GtkWidget      *drawing_area,
               GdkEventButton *event,
               gpointer        data)
@@ -619,6 +619,7 @@ palette_press (GtkWidget      *drawing_area,
     }
 
   gtk_widget_queue_clear (priv->last_palette);
+  return TRUE;
 }
 
 static void
@@ -793,9 +794,10 @@ palette_activate (GtkWidget   *widget,
     {
       priv = colorsel->private_data;
       palette_set_color (widget, GTK_COLOR_SELECTION (data), priv->color);
+      return TRUE;
     }
   
-  return TRUE;
+  return FALSE;
 }
 
 static GtkWidget*
@@ -811,6 +813,7 @@ palette_new (GtkColorSelection *colorsel)
   gtk_object_set_data (GTK_OBJECT (retval), "color_set", GINT_TO_POINTER (0)); 
   gtk_widget_set_events (retval, GDK_BUTTON_PRESS_MASK | GDK_EXPOSURE_MASK);
   
+
   gtk_signal_connect (GTK_OBJECT (retval), "expose_event",
                       GTK_SIGNAL_FUNC (palette_expose), colorsel);
   gtk_signal_connect (GTK_OBJECT (retval), "button_press_event",
@@ -907,15 +910,15 @@ mouse_motion (GtkWidget      *button,
   grab_color_at_mouse (button, event->x_root, event->y_root, data); 
 }
 
-static void
+static gboolean
 mouse_release (GtkWidget      *button,
               GdkEventButton *event,
               gpointer        data)
 {
   GtkColorSelection *colorsel = data;
   ColorSelectionPrivate *priv;
-  priv = colorsel->private_data;
-  
+  priv = colorsel->private_data;  
+
   gtk_signal_disconnect_by_func (GTK_OBJECT (button),
                                  GTK_SIGNAL_FUNC (mouse_motion), data);
   gtk_signal_disconnect_by_func (GTK_OBJECT (button),
@@ -923,10 +926,11 @@ mouse_release (GtkWidget      *button,
   
   grab_color_at_mouse (button, event->x_root, event->y_root, data);
   gdk_pointer_ungrab (0);
+  return TRUE;
 }
 
 /* Helper Functions */
-static void
+static gboolean
 mouse_press (GtkWidget      *button,
             GdkEventButton *event,
             gpointer        data)
@@ -934,7 +938,7 @@ mouse_press (GtkWidget      *button,
   GtkColorSelection *colorsel = data;
   ColorSelectionPrivate *priv;
   priv = colorsel->private_data;
-  
+
   gtk_signal_connect (GTK_OBJECT (button), "motion_notify_event",
                       GTK_SIGNAL_FUNC (mouse_motion),
                       data);
@@ -943,7 +947,8 @@ mouse_press (GtkWidget      *button,
                       data);
   gtk_signal_disconnect_by_func (GTK_OBJECT (button),
                                  GTK_SIGNAL_FUNC (mouse_press),
-                                 data); 
+                                 data);
+  return TRUE;
 }
 
 /* when the button is clicked */
@@ -966,6 +971,7 @@ get_screen_color (GtkWidget *button)
   if (priv->moving_dropper == FALSE)
     {
       priv->moving_dropper = TRUE;
+
       gtk_signal_connect (GTK_OBJECT (button), "button_press_event",
                           GTK_SIGNAL_FUNC (mouse_press), colorsel); 
       
@@ -1155,12 +1161,15 @@ make_label_spinbutton (GtkColorSelection *colorsel,
     }
   gtk_object_set_data (GTK_OBJECT (adjust), "COLORSEL", colorsel);
   *spinbutton = gtk_spin_button_new (adjust, 10.0, 0);
+
   gtk_signal_connect (GTK_OBJECT (*spinbutton), "focus_in_event",
                       GTK_SIGNAL_FUNC (widget_focus_in), colorsel);
   gtk_signal_connect (GTK_OBJECT (adjust), "value_changed",
                       GTK_SIGNAL_FUNC (adjustment_changed),
                       GINT_TO_POINTER (channel_type));
-  label = gtk_label_new (text);
+  label = gtk_label_new_with_mnemonic (text);
+  gtk_label_set_mnemonic_widget (GTK_LABEL (label), *spinbutton);
+
   gtk_misc_set_alignment (GTK_MISC (label), 1.0, 0.5);
   gtk_table_attach_defaults (GTK_TABLE (table), label, i, i+1, j, j+1);
   gtk_table_attach_defaults (GTK_TABLE (table), *spinbutton, i+1, i+2, j, j+1);
@@ -1402,20 +1411,22 @@ gtk_color_selection_init (GtkColorSelection *colorsel)
   gtk_table_set_row_spacings (GTK_TABLE (table), 4);
   gtk_table_set_col_spacings (GTK_TABLE (table), 4);
   
-  make_label_spinbutton (colorsel, &priv->hue_spinbutton, _("Hue:"), table, 0, 0, COLORSEL_HUE);
-  make_label_spinbutton (colorsel, &priv->sat_spinbutton, _("Saturation:"), table, 0, 1, COLORSEL_SATURATION);
-  make_label_spinbutton (colorsel, &priv->val_spinbutton, _("Value:"), table, 0, 2, COLORSEL_VALUE);
-  make_label_spinbutton (colorsel, &priv->red_spinbutton, _("Red:"), table, 6, 0, COLORSEL_RED);
-  make_label_spinbutton (colorsel, &priv->green_spinbutton, _("Green:"), table, 6, 1, COLORSEL_GREEN);
-  make_label_spinbutton (colorsel, &priv->blue_spinbutton, _("Blue:"), table, 6, 2, COLORSEL_BLUE);
+  make_label_spinbutton (colorsel, &priv->hue_spinbutton, _("_Hue:"), table, 0, 0, COLORSEL_HUE);
+  make_label_spinbutton (colorsel, &priv->sat_spinbutton, _("_Saturation:"), table, 0, 1, COLORSEL_SATURATION);
+  make_label_spinbutton (colorsel, &priv->val_spinbutton, _("_Value:"), table, 0, 2, COLORSEL_VALUE);
+  make_label_spinbutton (colorsel, &priv->red_spinbutton, _("_Red:"), table, 6, 0, COLORSEL_RED);
+  make_label_spinbutton (colorsel, &priv->green_spinbutton, _("_Green:"), table, 6, 1, COLORSEL_GREEN);
+  make_label_spinbutton (colorsel, &priv->blue_spinbutton, _("_Blue:"), table, 6, 2, COLORSEL_BLUE);
   gtk_table_attach_defaults (GTK_TABLE (table), gtk_hseparator_new (), 0, 8, 3, 4); 
   
-  priv->opacity_label = gtk_label_new (_("Opacity:")); 
+  priv->opacity_label = gtk_label_new_with_mnemonic (_("_Opacity:")); 
   gtk_misc_set_alignment (GTK_MISC (priv->opacity_label), 1.0, 0.5); 
   gtk_table_attach_defaults (GTK_TABLE (table), priv->opacity_label, 0, 1, 4, 5); 
   adjust = GTK_ADJUSTMENT (gtk_adjustment_new (0.0, 0.0, 255.0, 1.0, 1.0, 0.0)); 
   gtk_object_set_data (GTK_OBJECT (adjust), "COLORSEL", colorsel); 
-  priv->opacity_slider = gtk_hscale_new (adjust); 
+  priv->opacity_slider = gtk_hscale_new (adjust);
+  gtk_label_set_mnemonic_widget (GTK_LABEL (priv->opacity_label),
+                                 priv->opacity_slider);
   gtk_scale_set_draw_value (GTK_SCALE (priv->opacity_slider), FALSE);
   gtk_signal_connect (GTK_OBJECT(adjust), "value_changed",
                       GTK_SIGNAL_FUNC (adjustment_changed),
@@ -1429,12 +1440,16 @@ gtk_color_selection_init (GtkColorSelection *colorsel)
                       GTK_SIGNAL_FUNC (opacity_entry_changed), colorsel);
   gtk_table_attach_defaults (GTK_TABLE (table), priv->opacity_entry, 7, 8, 4, 5);
   
-  label = gtk_label_new (_("Hex Value:"));
+  label = gtk_label_new_with_mnemonic (_("He_x Value:"));
   gtk_table_attach_defaults (GTK_TABLE (table), label, 0, 1, 5, 6);
   gtk_misc_set_alignment (GTK_MISC (label), 1.0, 0.5);
   priv->hex_entry = gtk_entry_new ();
+
+  gtk_label_set_mnemonic_widget (GTK_LABEL (label), priv->hex_entry);
+
   gtk_signal_connect (GTK_OBJECT (priv->hex_entry), "activate",
                       GTK_SIGNAL_FUNC (hex_changed), colorsel);
+
   gtk_widget_set_usize (priv->hex_entry, 75, -1);  
   gtk_table_set_col_spacing (GTK_TABLE (table), 3, 15);
   gtk_table_attach_defaults (GTK_TABLE (table), priv->hex_entry, 1, 5, 5, 6);
@@ -1456,8 +1471,11 @@ gtk_color_selection_init (GtkColorSelection *colorsel)
   vbox = gtk_vbox_new (FALSE, 4);
   gtk_container_add (GTK_CONTAINER (priv->palette_frame), vbox);
   gtk_box_pack_start (GTK_BOX (vbox), table, FALSE, FALSE, 0);
-  button = gtk_button_new_with_label (_("Set Color"));
+
+  button = gtk_button_new_with_mnemonic (_("Set _Color"));
+
   gtk_signal_connect (GTK_OBJECT (button), "clicked", GTK_SIGNAL_FUNC (add_button_pressed), colorsel);
+
   gtk_box_pack_start (GTK_BOX (vbox), button, FALSE, FALSE, 0);
   gtk_container_set_border_width (GTK_CONTAINER (vbox), 4);
   
index be607894ce9f4af7f7f67f2d755d9ea7e8cde6a1..0bb3fc8aba915cf663241900d8a856203ca4a6af 100644 (file)
@@ -71,7 +71,7 @@ static void         gtk_combo_get_pos            (GtkCombo         *combo,
 static void         gtk_combo_popup_list         (GtkCombo         *combo);
 static void         gtk_combo_activate           (GtkWidget        *widget,
                                                  GtkCombo         *combo);
-static void         gtk_combo_popup_button_press (GtkWidget        *button,
+static gboolean     gtk_combo_popup_button_press (GtkWidget        *button,
                                                  GdkEventButton   *event,
                                                  GtkCombo         *combo);
 static void         gtk_combo_popup_button_leave (GtkWidget        *button,
@@ -491,7 +491,7 @@ gtk_combo_activate (GtkWidget        *widget,
                    NULL, NULL, GDK_CURRENT_TIME);
 }
 
-static void        
+static gboolean
 gtk_combo_popup_button_press (GtkWidget        *button,
                              GdkEventButton   *event,
                              GtkCombo         *combo)
@@ -510,6 +510,7 @@ gtk_combo_popup_button_press (GtkWidget        *button,
                    GDK_BUTTON_RELEASE_MASK,
                    NULL, NULL, event->time);
   gtk_grab_add (combo->list);
+  return TRUE;
 }
 
 static void         
@@ -628,7 +629,7 @@ gtk_combo_button_release (GtkWidget * widget, GdkEvent * event, GtkCombo * combo
                            GDK_BUTTON_RELEASE_MASK |
                            GDK_POINTER_MOTION_MASK, 
                            NULL, NULL, GDK_CURRENT_TIME);
-         return FALSE;
+         return TRUE;
        }
     }
   else
index ff4fe0d89596064a585341729b010301755e7377..2ea6d473c7036abdb87a70465daa76b7a1e2fa65 100644 (file)
@@ -824,9 +824,10 @@ gtk_ctree_button_press (GtkWidget      *widget,
          else
            gtk_ctree_expand (ctree, work);
 
-         return FALSE;
+         return TRUE;
        }
     }
+  
   return GTK_WIDGET_CLASS (parent_class)->button_press_event (widget, event);
 }
 
index 8954d9e56be1654467bc2952428d1d0757602052..e846dd117d67dec0857c41e2313b9ae65e69722b 100644 (file)
@@ -433,6 +433,7 @@ gtk_curve_graph_events (GtkWidget *widget, GdkEvent *event, GtkCurve *c)
   gfloat rx, ry, min_x;
   guint distance;
   gint x1, x2, y1, y2;
+  gint retval = FALSE;
 
   w = GTK_WIDGET (c);
   width = w->allocation.width - RADIUS * 2;
@@ -519,6 +520,7 @@ gtk_curve_graph_events (GtkWidget *widget, GdkEvent *event, GtkCurve *c)
          break;
        }
       gtk_curve_draw (c, width, height);
+      retval = TRUE;
       break;
 
     case GDK_BUTTON_RELEASE:
@@ -554,6 +556,7 @@ gtk_curve_graph_events (GtkWidget *widget, GdkEvent *event, GtkCurve *c)
        }
       new_type = GDK_FLEUR;
       c->grab_point = -1;
+      retval = TRUE;
       break;
 
     case GDK_MOTION_NOTIFY:
@@ -652,12 +655,14 @@ gtk_curve_graph_events (GtkWidget *widget, GdkEvent *event, GtkCurve *c)
          gdk_window_set_cursor (w->window, cursor);
          gdk_cursor_destroy (cursor);
        }
+      retval = TRUE;
       break;
 
     default:
       break;
     }
-  return FALSE;
+
+  return retval;
 }
 
 void
index 98772134527b0e74d7be567feaa366cd218753a4..982facda479337af787790d2119423dde218fe1f 100644 (file)
@@ -2466,6 +2466,7 @@ gtk_drag_source_event_cb (GtkWidget      *widget,
                          gpointer        data)
 {
   GtkDragSourceSite *site;
+  gboolean retval = FALSE;
   site = (GtkDragSourceSite *)data;
 
   switch (event->type)
@@ -2476,6 +2477,7 @@ gtk_drag_source_event_cb (GtkWidget      *widget,
          site->state |= (GDK_BUTTON1_MASK << (event->button.button - 1));
          site->x = event->button.x;
          site->y = event->button.y;
+         retval = TRUE;
        }
       break;
       
@@ -2483,6 +2485,7 @@ gtk_drag_source_event_cb (GtkWidget      *widget,
       if ((GDK_BUTTON1_MASK << (event->button.button - 1)) & site->start_button_mask)
        {
          site->state &= ~(GDK_BUTTON1_MASK << (event->button.button - 1));
+         retval = TRUE;
        }
       break;
       
@@ -2524,7 +2527,7 @@ gtk_drag_source_event_cb (GtkWidget      *widget,
                    gtk_drag_set_icon_default (context);
                }
 
-             return TRUE;
+             retval = TRUE;
            }
        }
       break;
@@ -2532,7 +2535,8 @@ gtk_drag_source_event_cb (GtkWidget      *widget,
     default:                   /* hit for 2/3BUTTON_PRESS */
       break;
     }
-  return FALSE;
+  
+  return retval;
 }
 
 static void 
index e07e08d45101dc4a750e7d044b59d359d8ec5e05..046980492aaada9966cbd474805143466bb560d4 100644 (file)
@@ -1348,7 +1348,7 @@ gtk_entry_button_release (GtkWidget      *widget,
   
   gtk_entry_update_primary_selection (entry);
              
-  return FALSE;
+  return TRUE;
 }
 
 static gint
index 130a81eab7c4b3c62b25f78aef84b5761fdc8d03..f2107d9acafb552a2c2bb4ba2b6f5ee431224fc3 100644 (file)
@@ -1058,13 +1058,14 @@ gtk_file_selection_create_dir (GtkWidget *widget,
                     FALSE, FALSE, 0);
   gtk_widget_show( vbox);
   
-  label = gtk_label_new (_("Directory name:"));
+  label = gtk_label_new_with_mnemonic (_("_Directory name:"));
   gtk_misc_set_alignment(GTK_MISC (label), 0.0, 0.0);
   gtk_box_pack_start (GTK_BOX (vbox), label, FALSE, FALSE, 5);
   gtk_widget_show (label);
 
   /*  The directory entry widget  */
   fs->fileop_entry = gtk_entry_new ();
+  gtk_label_set_mnemonic_widget (GTK_LABEL (label), fs->fileop_entry);
   gtk_box_pack_start (GTK_BOX (vbox), fs->fileop_entry, 
                      TRUE, TRUE, 5);
   GTK_WIDGET_SET_FLAGS (fs->fileop_entry, GTK_CAN_DEFAULT);
index 027b052a59d63f0dc5227825e77c96e3ee11204b..86e6ea4f003fbdc3284ff8db39859df79482f438 100644 (file)
@@ -186,23 +186,7 @@ gtk_font_selection_init(GtkFontSelection *fontsel)
   gtk_widget_show (table);
   gtk_table_set_col_spacings(GTK_TABLE(table), 8);
   gtk_box_pack_start (GTK_BOX (fontsel), table, TRUE, TRUE, 0);
-  
-  fontsel->font_label = gtk_label_new(_("Family:"));
-  gtk_misc_set_alignment (GTK_MISC (fontsel->font_label), 0.0, 0.5);
-  gtk_widget_show (fontsel->font_label);
-  gtk_table_attach (GTK_TABLE (table), fontsel->font_label, 0, 1, 0, 1,
-                   GTK_FILL, 0, 0, 0);
-  label = gtk_label_new(_("Style:"));
-  gtk_misc_set_alignment (GTK_MISC (label), 0.0, 0.5);
-  gtk_widget_show (label);
-  gtk_table_attach (GTK_TABLE (table), label, 1, 2, 0, 1,
-                   GTK_FILL, 0, 0, 0);
-  label = gtk_label_new(_("Size:"));
-  gtk_misc_set_alignment (GTK_MISC (label), 0.0, 0.5);
-  gtk_widget_show (label);
-  gtk_table_attach (GTK_TABLE (table), label, 2, 3, 0, 1,
-                   GTK_FILL, 0, 0, 0);
-  
+
   fontsel->font_entry = gtk_entry_new ();
   gtk_entry_set_editable(GTK_ENTRY(fontsel->font_entry), FALSE);
   gtk_widget_set_usize (fontsel->font_entry, 20, -1);
@@ -226,6 +210,29 @@ gtk_font_selection_init(GtkFontSelection *fontsel)
                      (GtkSignalFunc) gtk_font_selection_size_key_press,
                      fontsel);
   
+  fontsel->font_label = gtk_label_new_with_mnemonic (_("_Family:"));
+  gtk_label_set_mnemonic_widget (GTK_LABEL (fontsel->font_label),
+                                 fontsel->font_entry);
+  gtk_misc_set_alignment (GTK_MISC (fontsel->font_label), 0.0, 0.5);
+  gtk_widget_show (fontsel->font_label);
+  gtk_table_attach (GTK_TABLE (table), fontsel->font_label, 0, 1, 0, 1,
+                   GTK_FILL, 0, 0, 0);  
+  label = gtk_label_new_with_mnemonic (_("_Style:"));
+  gtk_label_set_mnemonic_widget (GTK_LABEL (label),
+                                 fontsel->font_style_entry);
+  gtk_misc_set_alignment (GTK_MISC (label), 0.0, 0.5);
+  gtk_widget_show (label);
+  gtk_table_attach (GTK_TABLE (table), label, 1, 2, 0, 1,
+                   GTK_FILL, 0, 0, 0);
+  label = gtk_label_new_with_mnemonic (_("Si_ze:"));
+  gtk_label_set_mnemonic_widget (GTK_LABEL (label),
+                                 fontsel->size_entry);
+  gtk_misc_set_alignment (GTK_MISC (label), 0.0, 0.5);
+  gtk_widget_show (label);
+  gtk_table_attach (GTK_TABLE (table), label, 2, 3, 0, 1,
+                   GTK_FILL, 0, 0, 0);
+  
+  
   /* Create the clists  */
   fontsel->font_clist = gtk_clist_new (1);
   gtk_clist_column_titles_hide (GTK_CLIST (fontsel->font_clist));
@@ -716,6 +723,7 @@ gtk_font_selection_size_key_press (GtkWidget   *w,
          fontsel->font_desc->size = new_size;
          gtk_font_selection_load_font (fontsel);
        }
+      return TRUE;
     }
   
   return FALSE;
index 700185baf319f84517cb052f8ce9d3246c5ed1f8..220ee8ecee79cb41a459de916b62875018806e24 100644 (file)
@@ -399,12 +399,13 @@ button_clicked_callback (GtkWidget *w, gpointer data)
          gtk_box_pack_start (GTK_BOX (vbox), hbox, TRUE, TRUE, 2);
          gtk_widget_show (hbox);
          
-         label = gtk_label_new (_("Gamma value"));
+         label = gtk_label_new (_("_Gamma value"));
          gtk_box_pack_start (GTK_BOX (hbox), label, FALSE, FALSE, 2);
          gtk_widget_show (label);
          
          sprintf (buf, "%g", c->gamma);
          c->gamma_text = gtk_entry_new ();
+          gtk_label_set_mnemonic_widget (GTK_LABEL (label), c->gamma_text);
          gtk_entry_set_text (GTK_ENTRY (c->gamma_text), buf);
          gtk_box_pack_start (GTK_BOX (hbox), c->gamma_text, TRUE, TRUE, 2);
          gtk_widget_show (c->gamma_text);
index e4d0bcc85319e866cb2922df65a7987cbbbb028c..654c1c0aee146608597e94b8925f6ebac8bb2403 100644 (file)
@@ -25,6 +25,9 @@
 #include <math.h>
 #include "gtksignal.h"
 #include "gtkhsv.h"
+#include "gdk/gdkkeysyms.h"
+#include "gtkbindings.h"
+#include "gtkcontainer.h"
 
 /*
  * Modified by the GTK+ Team and others 1997-2000.  See the AUTHORS
@@ -66,6 +69,9 @@ typedef struct {
   
   /* Dragging mode */
   DragMode mode;
+
+  guint focus_on_ring : 1;
+  
 } HSVPrivate;
 
 \f
@@ -74,6 +80,7 @@ typedef struct {
 
 enum {
   CHANGED,
+  MOVE,
   LAST_SIGNAL
 };
 
@@ -96,6 +103,11 @@ static gint gtk_hsv_motion         (GtkWidget      *widget,
                                    GdkEventMotion *event);
 static gint gtk_hsv_expose         (GtkWidget      *widget,
                                    GdkEventExpose *event);
+static gboolean gtk_hsv_focus      (GtkContainer    *container,
+                                    GtkDirectionType direction);
+
+static void gtk_hsv_move           (GtkHSV          *hsv,
+                                    GtkDirectionType dir);
 
 static guint hsv_signals[LAST_SIGNAL];
 static GtkWidgetClass *parent_class;
@@ -127,7 +139,7 @@ gtk_hsv_get_type (void)
       (GtkClassInitFunc) NULL
     };
     
-    hsv_type = gtk_type_unique (GTK_TYPE_WIDGET, &hsv_info);
+    hsv_type = gtk_type_unique (GTK_TYPE_CONTAINER, &hsv_info);
   }
   
   return hsv_type;
@@ -139,12 +151,16 @@ gtk_hsv_class_init (GtkHSVClass *class)
 {
   GtkObjectClass *object_class;
   GtkWidgetClass *widget_class;
+  GtkContainerClass *container_class;
+  GtkHSVClass    *hsv_class;
+  GtkBindingSet *binding_set;
   
   object_class = (GtkObjectClass *) class;
   widget_class = (GtkWidgetClass *) class;
+  container_class = GTK_CONTAINER_CLASS (class);
+  hsv_class = GTK_HSV_CLASS (class);
   
-  parent_class = gtk_type_class (GTK_TYPE_WIDGET);
-  
+  parent_class = gtk_type_class (GTK_TYPE_WIDGET);  
   
   object_class->destroy = gtk_hsv_destroy;
   
@@ -159,6 +175,10 @@ gtk_hsv_class_init (GtkHSVClass *class)
   widget_class->motion_notify_event = gtk_hsv_motion;
   widget_class->expose_event = gtk_hsv_expose;
 
+  container_class->focus = gtk_hsv_focus;
+  
+  hsv_class->move = gtk_hsv_move;
+  
   hsv_signals[CHANGED] =
     gtk_signal_new ("changed",
                    GTK_RUN_FIRST,
@@ -166,6 +186,45 @@ gtk_hsv_class_init (GtkHSVClass *class)
                    GTK_SIGNAL_OFFSET (GtkHSVClass, changed),
                    gtk_marshal_VOID__VOID,
                    GTK_TYPE_NONE, 0);
+
+  hsv_signals[MOVE] =
+    gtk_signal_new ("move",
+                   GTK_RUN_LAST | GTK_RUN_ACTION,
+                   GTK_CLASS_TYPE (object_class),
+                   GTK_SIGNAL_OFFSET (GtkHSVClass, move),
+                   gtk_marshal_VOID__ENUM,
+                   GTK_TYPE_NONE, 1, GTK_TYPE_DIRECTION_TYPE);
+
+  binding_set = gtk_binding_set_by_class (class);
+
+  gtk_binding_entry_add_signal (binding_set, GDK_Up, 0,
+                                "move", 1,
+                                GTK_TYPE_ENUM, GTK_DIR_UP);
+  gtk_binding_entry_add_signal (binding_set, GDK_KP_Up, 0,
+                                "move", 1,
+                                GTK_TYPE_ENUM, GTK_DIR_UP);
+  
+  gtk_binding_entry_add_signal (binding_set, GDK_Down, 0,
+                                "move", 1,
+                                GTK_TYPE_ENUM, GTK_DIR_DOWN);
+  gtk_binding_entry_add_signal (binding_set, GDK_KP_Down, 0,
+                                "move", 1,
+                                GTK_TYPE_ENUM, GTK_DIR_DOWN);
+
+  
+  gtk_binding_entry_add_signal (binding_set, GDK_Right, 0,
+                                "move", 1,
+                                GTK_TYPE_ENUM, GTK_DIR_RIGHT);
+  gtk_binding_entry_add_signal (binding_set, GDK_KP_Right, 0,
+                                "move", 1,
+                                GTK_TYPE_ENUM, GTK_DIR_RIGHT);
+  
+  gtk_binding_entry_add_signal (binding_set, GDK_Left, 0,
+                                "move", 1,
+                                GTK_TYPE_ENUM, GTK_DIR_LEFT);
+  gtk_binding_entry_add_signal (binding_set, GDK_KP_Left, 0,
+                                "move", 1,
+                                GTK_TYPE_ENUM, GTK_DIR_LEFT);
 }
 
 /* Object initialization function for the HSV color selector */
@@ -178,6 +237,7 @@ gtk_hsv_init (GtkHSV *hsv)
   hsv->priv = priv;
   
   GTK_WIDGET_SET_FLAGS (hsv, GTK_NO_WINDOW);
+  GTK_WIDGET_SET_FLAGS (hsv, GTK_CAN_FOCUS);
   
   priv->h = 0.0;
   priv->s = 0.0;
@@ -268,7 +328,8 @@ gtk_hsv_realize (GtkWidget *widget)
   attr.height = widget->allocation.height;
   attr.wclass = GDK_INPUT_ONLY;
   attr.event_mask = gtk_widget_get_events (widget);
-  attr.event_mask |= (GDK_BUTTON_PRESS_MASK
+  attr.event_mask |= (GDK_KEY_PRESS_MASK
+                      | GDK_BUTTON_PRESS_MASK
                      | GDK_BUTTON_RELEASE_MASK
                      | GDK_POINTER_MOTION_MASK);
   
@@ -727,6 +788,9 @@ gtk_hsv_button_press (GtkWidget      *widget,
                         compute_v (hsv, x, y),
                         priv->s,
                         priv->v);
+
+      gtk_widget_grab_focus (widget);
+      priv->focus_on_ring = TRUE;
       
       return TRUE;
     }
@@ -740,6 +804,10 @@ gtk_hsv_button_press (GtkWidget      *widget,
       
       compute_sv (hsv, x, y, &s, &v);
       gtk_hsv_set_color (hsv, priv->h, s, v);
+
+      gtk_widget_grab_focus (widget);
+      priv->focus_on_ring = FALSE;
+      
       return TRUE;
     }
   
@@ -967,17 +1035,21 @@ paint_ring (GtkHSV      *hsv,
   g_free (buf);
   
   /* Draw ring outline */
-  
-  gdk_rgb_gc_set_foreground (priv->gc, 0x000000);
-  
-  gdk_draw_arc (drawable, priv->gc, FALSE,
-               -x, -y,
-               priv->size - 1, priv->size - 1,
-               0, 360 * 64);
-  gdk_draw_arc (drawable, priv->gc, FALSE,
-               -x + priv->ring_width - 1, -y + priv->ring_width - 1,
-               priv->size - 2 * priv->ring_width + 1, priv->size - 2 * priv->ring_width + 1,
-               0, 360 * 64);
+
+  if (GTK_WIDGET_HAS_FOCUS (hsv) &&
+      priv->focus_on_ring)
+    {
+      gdk_rgb_gc_set_foreground (priv->gc, 0x000000);
+      
+      gdk_draw_arc (drawable, priv->gc, FALSE,
+                    -x, -y,
+                    priv->size - 1, priv->size - 1,
+                    0, 360 * 64);
+      gdk_draw_arc (drawable, priv->gc, FALSE,
+                    -x + priv->ring_width - 1, -y + priv->ring_width - 1,
+                    priv->size - 2 * priv->ring_width + 1, priv->size - 2 * priv->ring_width + 1,
+                    0, 360 * 64);
+    }
 }
 
 /* Converts an HSV triplet to an integer RGB triplet */
@@ -1175,11 +1247,15 @@ paint_triangle (GtkHSV      *hsv,
   
   g_free (buf);
   
-  /* Draw triangle outline */
-  
-  gdk_rgb_gc_set_foreground (priv->gc, 0x000000);
+  /* Draw triangle focus outline */
+
+  if (GTK_WIDGET_HAS_FOCUS (hsv) &&
+      !priv->focus_on_ring)
+    {
+      gdk_rgb_gc_set_foreground (priv->gc, 0x000000);
   
-  gdk_draw_polygon (drawable, priv->gc, FALSE, points, 3);
+      gdk_draw_polygon (drawable, priv->gc, FALSE, points, 3);
+    }
   
   /* Draw value marker */
   
@@ -1271,6 +1347,63 @@ gtk_hsv_expose (GtkWidget      *widget,
   return FALSE;
 }
 
+static gboolean
+gtk_hsv_focus (GtkContainer    *container,
+               GtkDirectionType dir)
+{
+  GtkHSV *hsv;
+  HSVPrivate *priv;
+
+  hsv = GTK_HSV (container);
+  priv = hsv->priv;
+  
+  if (!GTK_WIDGET_DRAWABLE (container) ||
+      !GTK_WIDGET_IS_SENSITIVE (container))
+    return FALSE;
+
+  if (!GTK_WIDGET_HAS_FOCUS (hsv))
+    {
+      gtk_widget_grab_focus (GTK_WIDGET (hsv));
+      return TRUE;
+    }
+  
+  switch (dir)
+    {
+    case GTK_DIR_UP:
+      if (priv->focus_on_ring)
+        return FALSE;
+      else
+        priv->focus_on_ring = TRUE;
+      break;
+
+    case GTK_DIR_DOWN:
+      if (priv->focus_on_ring)
+        priv->focus_on_ring = FALSE;
+      else
+        return FALSE;
+      break;
+
+    case GTK_DIR_LEFT:
+    case GTK_DIR_TAB_BACKWARD:
+      if (priv->focus_on_ring)
+        return FALSE;
+      else
+        priv->focus_on_ring = TRUE;
+      break;
+
+    case GTK_DIR_RIGHT:
+    case GTK_DIR_TAB_FORWARD:
+      if (priv->focus_on_ring)
+        priv->focus_on_ring = FALSE;
+      else
+        return FALSE;
+      break;
+    }
+
+  gtk_widget_queue_draw (GTK_WIDGET (hsv));
+
+  return TRUE;
+}
 
 /**
  * gtk_hsv_new:
@@ -1510,3 +1643,79 @@ gtk_rgb_to_hsv (gdouble  r,
   if (v)
     *v = b;
 }
+
+static void
+gtk_hsv_move (GtkHSV          *hsv,
+              GtkDirectionType dir)
+{
+  HSVPrivate *priv;
+  gdouble hue, sat, val;
+  gint hx, hy, sx, sy, vx, vy; /* HSV vertices */
+  gint x, y; /* position in triangle */
+  
+  priv = hsv->priv;
+
+  hue = priv->h;
+  sat = priv->s;
+  val = priv->v;
+
+  compute_triangle (hsv, &hx, &hy, &sx, &sy, &vx, &vy);
+
+  x = floor (sx + (vx - sx) * priv->v + (hx - vx) * priv->s * priv->v + 0.5);
+  y = floor (sy + (vy - sy) * priv->v + (hy - vy) * priv->s * priv->v + 0.5);
+  
+  switch (dir)
+    {
+    case GTK_DIR_UP:
+      if (priv->focus_on_ring)
+        hue += 0.02;
+      else
+        {
+          y -= 1;
+          compute_sv (hsv, x, y, &sat, &val);
+        }
+      break;
+
+    case GTK_DIR_DOWN:
+      if (priv->focus_on_ring)
+        hue -= 0.02;
+      else
+        {
+          y += 1;
+          compute_sv (hsv, x, y, &sat, &val);
+        }
+      break;
+
+    case GTK_DIR_LEFT:
+      if (priv->focus_on_ring)
+        hue += 0.02;
+      else
+        {
+          x -= 1;
+          compute_sv (hsv, x, y, &sat, &val);
+        }
+      break;
+
+    case GTK_DIR_RIGHT:
+      if (priv->focus_on_ring)
+        hue -= 0.02;
+      else
+        {
+          x += 1;
+          compute_sv (hsv, x, y, &sat, &val);
+        }
+      break;
+
+    default:
+      /* we don't care about the tab directions */
+      break;
+    }
+
+  /* Wrap */
+  if (hue < 0.0)
+    hue = 1.0;
+  else if (hue > 1.0)
+    hue = 0.0;
+  
+  gtk_hsv_set_color (hsv, hue, sat, val);
+}
index cc0a08d5bb92cf99762ea2b4ea8dd3549f954074..6a980859359aa19d06af927fed8be9bd082dd967 100644 (file)
@@ -31,7 +31,7 @@
  * GTK+ at ftp://ftp.gtk.org/pub/gtk/.
  */
 
-#include <gtk/gtkwidget.h>
+#include <gtk/gtkcontainer.h>
 
 #ifdef __cplusplus
 extern "C" {
@@ -39,7 +39,7 @@ extern "C" {
 
 #define GTK_TYPE_HSV            (gtk_hsv_get_type ())
 #define GTK_HSV(obj)            (GTK_CHECK_CAST ((obj), GTK_TYPE_HSV, GtkHSV))
-#define GTK_HSV_CLASS(klass)    (GTK_CHECK_CLASS_CAST ((klass), GTK_TYPE_HSV, GtkHSV))
+#define GTK_HSV_CLASS(klass)    (GTK_CHECK_CLASS_CAST ((klass), GTK_TYPE_HSV, GtkHSVClass))
 #define GTK_IS_HSV(obj)         (GTK_CHECK_TYPE ((obj), GTK_TYPE_HSV))
 #define GTK_IS_HSV_CLASS(klass) (GTK_CHECK_CLASS_TYPE ((klass), GTK_TYPE_HSV))
 #define GTK_HSV_GET_CLASS(obj)  (GTK_CHECK_GET_CLASS ((obj), GTK_TYPE_HSV, GtkHSVClass))
@@ -50,7 +50,8 @@ typedef struct _GtkHSVClass GtkHSVClass;
 
 struct _GtkHSV
 {
-  GtkWidget parent_instance;
+  /* we derive from container purely so we can have a _focus method */
+  GtkContainer parent_instance;
   
   /* Private data */
   gpointer priv;
@@ -58,11 +59,15 @@ struct _GtkHSV
 
 struct _GtkHSVClass
 {
-  GtkWidgetClass parent_class;
+  GtkContainerClass parent_class;
   
   /* Notification signals */
   
   void (*changed) (GtkHSV *hsv);
+
+  /* Keybindings */
+  void (* move) (GtkHSV          *hsv,
+                 GtkDirectionType type);
 };
 
 
index e150f66d7ff5660566a6a215290f2f14a0289fba..ba0901fdd05701e2864f443a228119dcddb0acc8 100644 (file)
@@ -215,10 +215,11 @@ gtk_input_dialog_init (GtkInputDialog *inputd)
       util_box = gtk_hbox_new (FALSE, 2);
       gtk_box_pack_start (GTK_BOX (vbox), util_box, FALSE, FALSE, 0);
 
-      label = gtk_label_new(_("Device:"));
+      label = gtk_label_new(_("_Device:"));
       gtk_box_pack_start (GTK_BOX (util_box), label, FALSE, FALSE, 2);
 
       optionmenu = gtk_option_menu_new ();
+      gtk_label_set_mnemonic_widget (GTK_LABEL (label), optionmenu);
       gtk_box_pack_start (GTK_BOX (util_box), optionmenu, TRUE, TRUE, 2);
       gtk_widget_show (optionmenu);
       gtk_option_menu_set_menu (GTK_OPTION_MENU (optionmenu), device_menu);
@@ -255,10 +256,11 @@ gtk_input_dialog_init (GtkInputDialog *inputd)
                          (GtkSignalFunc) gtk_input_dialog_set_mapping_mode,
                          GINT_TO_POINTER (GDK_MODE_WINDOW));
 
-      label = gtk_label_new(_("Mode: "));
+      label = gtk_label_new(_("_Mode: "));
       gtk_box_pack_start (GTK_BOX (util_box), label, FALSE, FALSE, 2);
 
       inputd->mode_optionmenu = gtk_option_menu_new ();
+      gtk_label_set_mnemonic_widget (GTK_LABEL (label), inputd->mode_optionmenu);
       gtk_box_pack_start (GTK_BOX (util_box), inputd->mode_optionmenu, FALSE, FALSE, 2);
       gtk_widget_show (inputd->mode_optionmenu);
       gtk_option_menu_set_menu (GTK_OPTION_MENU (inputd->mode_optionmenu), mapping_menu);
@@ -285,7 +287,7 @@ gtk_input_dialog_init (GtkInputDialog *inputd)
       
       /*  The axis listbox  */
 
-      label = gtk_label_new (_("Axes"));
+      label = gtk_label_new (_("_Axes"));
 
       inputd->axis_listbox = gtk_scrolled_window_new (NULL, NULL);
       gtk_scrolled_window_set_policy (GTK_SCROLLED_WINDOW(inputd->axis_listbox),
@@ -301,7 +303,7 @@ gtk_input_dialog_init (GtkInputDialog *inputd)
 
       /* Keys listbox */
 
-      label = gtk_label_new (_("Keys"));
+      label = gtk_label_new (_("_Keys"));
 
       inputd->keys_listbox = gtk_scrolled_window_new (NULL, NULL);
       gtk_widget_set_usize (inputd->keys_listbox, KEYS_LIST_WIDTH, KEYS_LIST_HEIGHT);
index d784ab641a44a03de1ced3628d93c50ed81513a4..cded0cf6b6a6a0ea3a9b4b3ab2585652d8baf711 100644 (file)
@@ -776,6 +776,8 @@ gtk_list_button_press (GtkWidget      *widget,
        default:
          break;
        }
+
+      return TRUE;
     }
 
   return FALSE;
@@ -828,6 +830,8 @@ gtk_list_button_release (GtkWidget  *widget,
        default:
          break;
        }
+
+      return TRUE;
     }
   
   return FALSE;
index 7f1f6ece05e1438a684dab6ac7eaf3e6061f4507..e87adcbae30608678342d4aa315e19be7c488cd1 100644 (file)
@@ -451,8 +451,12 @@ gtk_list_item_button_press (GtkWidget      *widget,
   g_return_val_if_fail (event != NULL, FALSE);
 
   if (event->type == GDK_BUTTON_PRESS)
-    if (!GTK_WIDGET_HAS_FOCUS (widget))
-      gtk_widget_grab_focus (widget);
+    {
+      if (!GTK_WIDGET_HAS_FOCUS (widget))
+       gtk_widget_grab_focus (widget);
+
+      return TRUE;
+    }
 
   return FALSE;
 }
index cdd00988c342711cae32f5bee63cd9da1b773c06..58437991449ebe0714de719f0f795b8984e05f5d 100644 (file)
@@ -430,7 +430,7 @@ gtk_menu_shell_button_press (GtkWidget      *widget,
 
   if (menu_shell->parent_menu_shell)
     {
-      gtk_widget_event (menu_shell->parent_menu_shell, (GdkEvent*) event);
+      return gtk_widget_event (menu_shell->parent_menu_shell, (GdkEvent*) event);
     }
   else if (!menu_shell->active || !menu_shell->button)
     {
@@ -484,8 +484,7 @@ gtk_menu_shell_button_release (GtkWidget      *widget,
        {
          menu_shell->button = 0;
          if (menu_shell->parent_menu_shell)
-           gtk_widget_event (menu_shell->parent_menu_shell, (GdkEvent*) event);
-         return TRUE;
+           return gtk_widget_event (menu_shell->parent_menu_shell, (GdkEvent*) event);
        }
       
       menu_shell->button = 0;
index 1ee200d4d8b47fee3b53aa7342822f5ef348bdce..530b5de94948a048a0faa3fe9b74bf58e08c1d97 100644 (file)
@@ -1103,9 +1103,10 @@ gtk_notebook_button_press (GtkWidget      *widget,
        }
       if (!children && !GTK_WIDGET_HAS_FOCUS (widget))
        gtk_widget_grab_focus (widget);
+      return TRUE;
     }
    
-  return TRUE;
+  return FALSE;
 }
 
 static gint
@@ -1137,9 +1138,11 @@ gtk_notebook_button_release (GtkWidget      *widget,
       notebook->click_child = 0;
       notebook->button = 0;
       gtk_notebook_redraw_arrows (notebook);
-      
+
+      return TRUE;
     }
-  return FALSE;
+  else
+    return FALSE;
 }
 
 static gint
index bc7f272d399db79f611694d254b671ecf6399f44..999be5ac9ef786042c48c3678d35eac7911ab406 100644 (file)
@@ -892,9 +892,10 @@ gtk_spin_button_button_press (GtkWidget      *widget,
                }
              gtk_spin_button_draw_arrow (spin, GTK_ARROW_DOWN);
            }
+         return TRUE;
        }
       else
-       GTK_WIDGET_CLASS (parent_class)->button_press_event (widget, event);
+       return GTK_WIDGET_CLASS (parent_class)->button_press_event (widget, event);
     }
   return FALSE;
 }
@@ -954,9 +955,10 @@ gtk_spin_button_button_release (GtkWidget      *widget,
       spin->click_child = 2;
       spin->button = 0;
       gtk_spin_button_draw_arrow (spin, click_child);
+      return TRUE;
     }
   else
-    GTK_WIDGET_CLASS (parent_class)->button_release_event (widget, event);
+    return GTK_WIDGET_CLASS (parent_class)->button_release_event (widget, event);
 
   return FALSE;
 }
index 2d1815ad3de0d2bf982f207de7d7c2f630f0f5c6..66c1a4824e7f211a7f3da91976f7b9154db40200 100644 (file)
@@ -1729,7 +1729,7 @@ gtk_text_button_press (GtkWidget      *widget,
        }
     }
   
-  return FALSE;
+  return TRUE;
 }
 
 static gint
@@ -1790,7 +1790,7 @@ gtk_text_button_release (GtkWidget      *widget,
   find_cursor (text, TRUE);
   draw_cursor (text, FALSE);
   
-  return FALSE;
+  return TRUE;
 }
 
 static gint
@@ -1907,8 +1907,6 @@ gtk_text_key_press (GtkWidget   *widget,
   g_return_val_if_fail (GTK_IS_TEXT (widget), FALSE);
   g_return_val_if_fail (event != NULL, FALSE);
   
-  return_val = FALSE;
-  
   text = GTK_TEXT (widget);
   old_editable = GTK_OLD_EDITABLE (widget);
   
@@ -2094,8 +2092,6 @@ gtk_text_key_press (GtkWidget   *widget,
              
              return_val = TRUE;
            }
-         else
-           return_val = FALSE;
        }
       
       if (return_val && (old_editable->current_pos != initial_pos))
index 3e48be3343782999508a97d9df60c66657fa776e..938e442f6f3b45a080a7cca8479d630184aaa7c5 100644 (file)
@@ -1622,7 +1622,7 @@ gtk_text_tag_event (GtkTextTag        *tag,
                     GdkEvent          *event,
                     const GtkTextIter *iter)
 {
-  gint retval = FALSE;
+  gboolean retval = FALSE;
 
   g_return_val_if_fail (GTK_IS_TEXT_TAG (tag), FALSE);
   g_return_val_if_fail (GTK_IS_OBJECT (event_object), FALSE);
index 30e33f3d426adc14fc3158d3ccc1df19e9c3fd23..3ee94dd8d7315309c7b569a28bde44c49d58595d 100644 (file)
@@ -2846,7 +2846,7 @@ emit_event_on_tags (GtkWidget   *widget,
 {
   GSList *tags;
   GSList *tmp;
-  gint retval = FALSE;
+  gboolean retval = FALSE;
   GtkTextView *text_view;
 
   text_view = GTK_TEXT_VIEW (widget);
@@ -2924,7 +2924,7 @@ gtk_text_view_event (GtkWidget *widget, GdkEvent *event)
 static gint
 gtk_text_view_key_press_event (GtkWidget *widget, GdkEventKey *event)
 {
-  gint retval = FALSE;
+  gboolean retval = FALSE;
   GtkTextView *text_view = GTK_TEXT_VIEW (widget);
 
   if (text_view->layout == NULL ||
index 724118ddae5e54701eb31a3bcce9c2979941c05d..52a3f0b38ab5ab363798d8f8ab6fd843cfdf91c4 100644 (file)
@@ -486,8 +486,10 @@ gtk_tooltips_event_handler (GtkWidget *widget,
        tooltips->use_sticky_delay = use_sticky_delay;
       }
       break;
+      
     default:
       gtk_tooltips_set_active_widget (tooltips, NULL);
+      return TRUE;
       break;
     }
 
index 162161ac1677bb96ac7bab9959dabc37d9a1de80..15c3e2a2fda30e3c419bf29cf92b0b05b6388b7b 100644 (file)
@@ -84,7 +84,7 @@ static void gtk_real_tree_item_collapse (GtkTreeItem      *item);
 static void gtk_real_tree_item_expand   (GtkTreeItem      *item);
 static void gtk_real_tree_item_collapse (GtkTreeItem      *item);
 static void gtk_tree_item_destroy        (GtkObject *object);
-static void gtk_tree_item_subtree_button_click (GtkWidget *widget);
+static gint gtk_tree_item_subtree_button_click (GtkWidget *widget);
 static void gtk_tree_item_subtree_button_changed_state (GtkWidget *widget);
 
 static void gtk_tree_item_map(GtkWidget*);
@@ -172,7 +172,7 @@ gtk_tree_item_class_init (GtkTreeItemClass *class)
 }
 
 /* callback for event box mouse event */
-static void 
+static gint
 gtk_tree_item_subtree_button_click (GtkWidget *widget)
 {
   GtkTreeItem* item;
@@ -182,12 +182,14 @@ gtk_tree_item_subtree_button_click (GtkWidget *widget)
   
   item = (GtkTreeItem*) gtk_object_get_user_data (GTK_OBJECT (widget));
   if (!GTK_WIDGET_IS_SENSITIVE (item))
-    return;
+    return FALSE;
   
   if (item->expanded)
     gtk_tree_item_collapse (item);
   else
     gtk_tree_item_expand (item);
+
+  return TRUE;
 }
 
 /* callback for event box state changed */
@@ -711,7 +713,7 @@ gtk_tree_item_button_press (GtkWidget      *widget,
        && !GTK_WIDGET_HAS_FOCUS (widget))
       gtk_widget_grab_focus (widget);
 
-  return FALSE;
+  return (event->type == GDK_BUTTON_PRESS && GTK_WIDGET_IS_SENSITIVE(widget));
 }
 
 static gint
index 5f1453b83f012146b5cfbcd8bbd8319a42937c49..1c86b567f57864fd5f3aca0892ce0331e4f2d2e9 100644 (file)
@@ -1080,7 +1080,7 @@ gtk_tree_model_sort_func  (gconstpointer a,
   GValue value_a = {0, };
   GValue value_b = {0, };
   SortData *sort_data = user_data;
-  gint retval;
+  gboolean retval;
 
   gtk_tree_model_get_value (sort_data->model, (GtkTreeIter *) a, sort_data->sort_col, &value_a);
   gtk_tree_model_get_value (sort_data->model, (GtkTreeIter *) b, sort_data->sort_col, &value_b);
index 761ab3b641160faf4f85b6cc612d89e72f2127fd..fc9412b5df2fb45cbfba67e0f335028b4ce5c0e2 100644 (file)
@@ -3178,7 +3178,7 @@ gtk_tree_view_discover_dirty_iter (GtkTreeView *tree_view,
   GtkTreeViewColumn *column;
   GList *list;
   gint i;
-  gint retval = FALSE;
+  gboolean retval = FALSE;
   gint tmpheight;
 
   if (height)